Skip to content

ci: revert PRs #762 and #775#776

Merged
brettlangdon merged 2 commits intomainfrom
brettlangdon/revert.ddtrace-serverless
Apr 2, 2026
Merged

ci: revert PRs #762 and #775#776
brettlangdon merged 2 commits intomainfrom
brettlangdon/revert.ddtrace-serverless

Conversation

@brettlangdon
Copy link
Copy Markdown
Member

What does this PR do?

Reverts PRs #762 and #775

Motivation

These introduced a breaking change in the layer building step.

Testing Guidelines

Additional Notes

Types of Changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)

@brettlangdon brettlangdon requested review from a team as code owners April 2, 2026 13:15
@brettlangdon brettlangdon merged commit a3f0640 into main Apr 2, 2026
104 of 105 checks passed
@brettlangdon brettlangdon deleted the brettlangdon/revert.ddtrace-serverless branch April 2, 2026 13:39
emmettbutler added a commit that referenced this pull request Apr 2, 2026
emmettbutler added a commit that referenced this pull request Apr 2, 2026
* Revert "ci: revert PRs #762 and #775 (#776)"

This reverts commit a3f0640.

* try a different way of checking success

* refactor to only do docker build once

* oh yeah

* catch error code

* bite the proverbial bullet

* *shakes fist* ....bash!!!

* more handling

* debug output

* construct basename when given a wheel directly

* handle dotslash in basename
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants